Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adding net income loss fields #716

Closed
wants to merge 7 commits into from
Closed

Adding net income loss fields #716

wants to merge 7 commits into from

Conversation

tsugumi-sys
Copy link

close #715

@tsugumi-sys
Copy link
Author

@justinpolygon
Hey, could you review this, will it take much longer for the review???

@justinpolygon justinpolygon marked this pull request as draft August 27, 2024 18:02
@justinpolygon
Copy link
Contributor

Sorry for all the notifications here. I'm just making sure all the fields match and then getting the tests to run correctly. Move to draft to stop all the notifications while we iron things out.

@justinpolygon
Copy link
Contributor

Hey @tsugumi-sys, thanks for the PR. I ended up putting together a much large fix that aligns with how we use the other APIs. I'm going to close this PR but this is fixed now via #817 and is in the latest release https://github.com/polygon-io/client-python/releases/tag/v1.14.3.

You should be able to just run pip install -U polygon-api-client to get the latest update.

Here's an example for fetching diluted_earnings_per_share and net_income_loss:

from polygon import RESTClient

client = RESTClient()  # POLYGON_API_KEY environment variable is used

financials = []
for f in client.vx.list_stock_financials("AAPL", filing_date="2024-11-01"):
    financials.append(f)

    # get diluted_earnings_per_share
    print(f.financials.income_statement.diluted_earnings_per_share)

    # get net_income_loss
    print(f.financials.income_statement.net_income_loss)

Here's the output:

$ python3 examples/rest/stocks-stock_financials.py

DataPoint(
    label="Diluted Earnings Per Share",
    order=4300,
    unit="USD / shares",
    value=6.08,
    derived_from=None,
    formula=None,
    source=None,
    xpath=None,
)
DataPoint(
    label="Net Income/Loss",
    order=3200,
    unit="USD",
    value=93736000000.0,
    derived_from=None,
    formula=None,
    source=None,
    xpath=None,
)
DataPoint(
    label="Diluted Earnings Per Share",
    order=4300,
    unit="USD / shares",
    value=6.08,
    derived_from=None,
    formula=None,
    source=None,
    xpath=None,
)
DataPoint(
    label="Net Income/Loss",
    order=3200,
    unit="USD",
    value=93736000000.0,
    derived_from=None,
    formula=None,
    source=None,
    xpath=None,
)

@tsugumi-sys
Copy link
Author

tsugumi-sys commented Jan 11, 2025 •
edited
Loading

@justinpolygon
Thank you very much！

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbonzo jbonzo Awaiting requested review from jbonzo

@mmoghaddam385 mmoghaddam385 Awaiting requested review from mmoghaddam385

@justinpolygon justinpolygon Awaiting requested review from justinpolygon justinpolygon is a code owner

@penelopus penelopus Awaiting requested review from penelopus penelopus will be requested when the pull request is marked ready for review penelopus is a code owner

@davidwf-polygonio davidwf-polygonio Awaiting requested review from davidwf-polygonio davidwf-polygonio will be requested when the pull request is marked ready for review davidwf-polygonio is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

I want to add net_income_loss field for stock financials vx api
2 participants
@tsugumi-sys @justinpolygon